feat(web): Show user chat history#721
Conversation
|
Note Other AI code review bot(s) detectedCodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review. WalkthroughThe chat page now fetches authenticated users' chat history, validates it, and renders a new horizontal resizable two-panel layout with a chat sidebar and main chat area; minor spacing tweak in the search landing page and changelog updated with an "ask sidebar" entry. Changes
Sequence DiagramsequenceDiagram
participant Page as Chat Page
participant Auth as auth()
participant Service as getUserChatHistory()
participant Layout as ResizablePanelGroup
participant Sidebar as ChatSidePanel
participant Main as Main Chat Panel
Page->>Auth: request auth state
Auth-->>Page: return auth state
alt authenticated
Page->>Service: fetch user chat history
Service-->>Page: return chatHistory
alt chatHistory is error
Page->>Page: throw ServiceErrorException
else valid chatHistory
Page->>Layout: render resizable layout
Layout->>Sidebar: provide chatHistory + auth state
Sidebar-->>Layout: render history UI
Layout->>Main: render main chat area (LandingPageChatBox, logo, carousel, demos)
Main-->>Layout: render complete
end
else unauthenticated
Page->>Layout: render layout without chatHistory
Layout->>Sidebar: render empty sidebar
Layout->>Main: render main chat area
end
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Hey @Nil2000 thanks for the PR. 
Instead of having cards, I think we would be better off re-using the existing chat history sidebar. This is a more common UX pattern, as seen in tools like claude: |
|
@brendan-kellam I believe this would do |
Nice I think this is a better. There are definitely improvements we can make to the sidebar (using the shadcn sidebar component comes to mind as it has a better UX interaction pattern imo), but we can leave that for the future. |
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
This PR is being reviewed by Cursor Bugbot
Details
You are on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle.
To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.
|
@brendan-kellam any update on this? |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@packages/web/src/app/`[domain]/chat/page.tsx:
- Line 95: The Tailwind class "md:mt-18" in the JSX div with classes "flex
flex-col justify-center items-center mt-8 mb-8 md:mt-18 w-full px-5" is invalid
and will be ignored; update that class to "md:mt-16" in the same div (page.tsx)
so the medium-screen top margin uses the project's valid spacing scale.
|
Thanks @Nil2000 ! |
2 participants
Fixes : #660
Implemented recent chats
For now I have set to 3
Note
Introduces a resizable chat sidebar with recent history on the Ask homepage and wires up authenticated chat history retrieval.
ChatSidePanel,ResizablePanelGroup,ResizablePanel, andAnimatedResizableHandletochat/page.tsxfor a collapsible, resizable layoutsessionviaauth()and loadsgetUserChatHistory(); surfaces service errors withServiceErrorExceptionCHANGELOG.mdto note the new ask sidebarWritten by Cursor Bugbot for commit 4344a18. This will update automatically on new commits. Configure here.
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.